1589923 - Strip https:// instead of http:// from URLs in the results view

Status: RESOLVED FIXED

(Firefox :: Address Bar, enhancement, P2)

Description

[Dão Gottwald [:dao]]

Verdi mentioned that in order to display more meaningful content in the view, we should look into hiding https:// there. Historically we didn't do that so that the user would know beforehand whether they'll go to an encrypted site. Now that most sites use https, I think it makes sense to flip that around, i.e. show http:// and other protocols but hide https://. In the input we'll still show https:// and hide http://. Changing that in the future is an option but will require more invasive changes to URI fixup.

Comment 1

[Dão Gottwald [:dao]]

Attachment: Bug 1589923 - Strip https:// instead of http:// from URLs in the results view. r=mak

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/675c7c3794bc
Strip https:// instead of http:// from URLs in the results view. r=mak

Comment 3

[Stefan Hindli [:stefan_hindli]]

Backed out changeset 675c7c3794bc (Bug 1589923) for causing xpcshell failures in xpcshell.ini:browser/components/extensions/test/xpcshell/test_ext_urlbar.js CLOSED TREE

https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&selectedJob=272264980

https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=272264980&repo=autoland&lineNumber=2776

Comment 4

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4b0f6bda1823
Backed out changeset 675c7c3794bc for causing xpcshell failures in xpcshell.ini:browser/components/extensions/test/xpcshell/test_ext_urlbar.js CLOSED TREE

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bc06b3f5a1f9
Strip https:// instead of http:// from URLs in the results view. r=mak

Comment 6

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/bc06b3f5a1f9